/retest

Do these changes require a newer version of pulp? Because the pods are failing with:
/usr/local/bin/start-api: line 33: /usr/local/bin/pulpcore-api: No such file or directory or
/usr/local/bin/start-content-app: line 12: /usr/local/bin/pulpcore-content: No such file or directory

Do these changes require a newer version of pulp? Because the pods are failing with: /usr/local/bin/start-api: line 33: /usr/local/bin/pulpcore-api: No such file or directory or /usr/local/bin/start-content-app: line 12: /usr/local/bin/pulpcore-content: No such file or directory

those changes are needed because of f80ef95#diff-60f61ab7a8d1910d86d9fda2261620314edcae5894d5aaa236b821c7256badd7R115

However, the cli path is probably wrong in this PR because pulpcore should be installed in the virtual environmant under /venv right ?

I just updated the PR with the correct path to pulpcore-api. It now fails with this error in the test playbook check.

setuptools.extern.packaging.version.InvalidVersion: Invalid version: ''"

Maybe we need to re-trigger CI to see if it is flake? I don't have permission to do so.

/retest

@rochacbruno it was failing when deploying with the galaxy-operator with this error:

Error: No such option: --limit-request-field_size (Possible options: --limit-request-field-size, --limit-request-fields, --limit-request-line)

So with the new changes, it needs to be:

--limit-request-field-size 32768

as per:

• https://github.com/pulp/pulpcore/pull/4249/files#diff-7cc0f6c8751b0c76e78bf692d9d6a19fd3c4a20715aa88047bfdc6c42af151c3R115

This latest push should fix it.

I just built a galaxy-ng image from this branch and tested it out with the operator and it now works:

NAME                                                  READY   STATUS    RESTARTS   AGE
galaxy-api-69bdf7b687-k5h6d                           1/1     Running   0          12m
galaxy-content-5669c9558f-fnsll                       1/1     Running   0          13m
galaxy-content-5669c9558f-wrxpr                       1/1     Running   0          13m
galaxy-operator-controller-manager-657f4b5466-rrfdf   2/2     Running   0          14m
galaxy-postgres-13-0                                  1/1     Running   0          13m
galaxy-redis-54b6b744db-wqwxd                         1/1     Running   0          12m
galaxy-web-55bf78bcbd-wc45v                           1/1     Running   0          13m
galaxy-worker-869f4987c9-m9pcz                        1/1     Running   0          12m

Caveat: I had to remove this flag because the pulpcore-api script didn't allow it, I think the pulp team may need to add a click option for this, I'll talk with them about this.

--limit-request-field-size 32768

@rochacbruno I think we should merge this PR as is. I expect the tests to pass shortly.

While testing this, I ran in to another bug, pulpcore-api does not correctly handle the --limit-request-field_size flag for gunicorn. This is a simple fix in pulpcore, but will require them to cut a new release and for the galaxy_ng project to bump the pulpcore dependency. So once that new pulpcore versionis available, in a separate PR, we should:

• bump the pulpcore version to the latest with the fix
• remove the comment and add back the --limit-request-field_size flag